packages/uniwind/src/core/web/cssListener.ts (1)

53-58: Returning a no-op from inside forEach is dead code.

Array.prototype.forEach ignores callback return values, so return () => {} on line 57 just exits the current iteration; the empty function is never used. A bare return is equivalent and clearer.

♻️ Proposed tweak

             if (!mediaQuery) {
-                return () => {}
+                return
             }

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@packages/uniwind/src/core/web/cssListener.ts` around lines 53 - 58, The
forEach callback in cssListener.ts is returning a no-op function which is never
used because Array.prototype.forEach ignores return values; update the loop in
the method that calls classNames.split(' ') so that when
classNameMediaQueryListeners.get(className) is falsy you use a bare return to
skip the iteration (or switch to a for...of and use continue/collect listeners
as needed), and ensure any intended listener functions are actually
collected/returned by referencing classNameMediaQueryListeners and the
surrounding method name so the empty function is not erroneously produced.

packages/uniwind/.oxlintrc.json (2)

17-28: ignorePatterns doesn't cover **/*.d.ts broadly.

The individual *.d.ts entries (types.d.ts, no-types.d.ts, src/metro/index.d.ts, src/vite/index.d.ts) suggest intent to skip declaration files. A single **/*.d.ts pattern would cover current + future declaration files and is easier to maintain. Optional.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@packages/uniwind/.oxlintrc.json` around lines 17 - 28, The ignorePatterns
currently lists specific declaration files (types.d.ts, no-types.d.ts,
src/metro/index.d.ts, src/vite/index.d.ts) which is brittle; update the
"ignorePatterns" array in .oxlintrc.json to replace those individual entries
with a single glob pattern "**/*.d.ts" so all current and future TypeScript
declaration files are ignored consistently while keeping other entries (dist,
specs, jest.config.js, tests, babel.config.cjs) unchanged.

---

15-15: Confirm intent of disabling typescript/no-implied-eval globally.

Disabling no-implied-eval repo-wide silences a real security signal (setTimeout("code"), new Function(...) etc.). If it's being turned off to suppress a specific false positive, consider scoping it with inline directives instead.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@packages/uniwind/.oxlintrc.json` at line 15, You've globally disabled the
typescript/no-implied-eval rule which mutes an important security check; either
confirm this intent or revert the global rule and instead apply scoped
exceptions where necessary by leaving the rule enabled and adding per-file or
per-line disables for the specific false positives (use inline
eslint-disable-next-line or eslint-disable comments targeting
typescript/no-implied-eval) or change the config to "warn" rather than "off";
update the .oxlintrc.json entry for typescript/no-implied-eval accordingly and
remove the global "off" if you choose scoped suppressions.

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes